Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @dan-starkware and @ShahakShama)

crates/papyrus_network/src/streamed_data_protocol/handler.rs line 116 at r1 (raw file):

        cx: &mut Context<'_>,
    ) -> bool {
        let Poll::Ready(finish_reason) = inbound_session.poll_unpin(cx) else {

I don't see any added value to the short circuit here since it's in an if statement any way.
I suggest to improve readability by defining a variable is_session_alive and return it in the end

crates/papyrus_network/src/streamed_data_protocol/handler.rs line 166 at r1 (raw file):

        // Handle inbound sessions.
        self.id_to_inbound_session.retain(|inbound_session_id, inbound_session| {
            if Self::poll_inbound_session(

same here the short circuiting is confusing to read especially since it's nested function

Reviewed 2 of 2 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @dan-starkware)

Reviewed 1 of 1 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @dan-starkware)